Skip to content

fix(Button): fix accessibility issues - #6870

Merged
jsulpis merged 2 commits into
mainfrom
button-accessibility
Sep 4, 2026
Merged

fix(Button): fix accessibility issues#6870
jsulpis merged 2 commits into
mainfrom
button-accessibility

Conversation

@jsulpis

@jsulpis jsulpis commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

Fix Button accessibility issues. One remaining issue is caused by the colors from the base theme which has a bad contrast.

@changeset-bot

changeset-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5dd0018

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@ultraviolet/ui Patch
@ultraviolet/form Patch
@ultraviolet/nextjs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.97%. Comparing base (89671b5) to head (a2ede52).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           button-fix    #6870   +/-   ##
===========================================
  Coverage       90.97%   90.97%           
===========================================
  Files             572      572           
  Lines           10526    10526           
  Branches         3797     3819   +22     
===========================================
  Hits             9576     9576           
  Misses            950      950           
Files with missing lines Coverage Δ
packages/ui/src/components/Button/index.tsx 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89671b5...a2ede52. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jsulpis
jsulpis requested a review from a team as a code owner September 4, 2026 07:21
@jsulpis
jsulpis force-pushed the button-accessibility branch from 1110a84 to a2ede52 Compare September 4, 2026 07:30
Base automatically changed from button-fix to main September 4, 2026 11:43
@jsulpis
jsulpis force-pushed the button-accessibility branch from a2ede52 to 5dd0018 Compare September 4, 2026 11:45
@jsulpis
jsulpis merged commit 05e6ecd into main Sep 4, 2026
10 checks passed
@jsulpis
jsulpis deleted the button-accessibility branch September 4, 2026 12:00
@github-actions github-actions Bot mentioned this pull request Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accessibility enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants